Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[WIP] Replace ECS with Harmony #613

Draft
wants to merge 20 commits into
base: low-hanging-refactors
Choose a base branch
from
Draft

[WIP] Replace ECS with Harmony #613

wants to merge 20 commits into from

Conversation

mfisher87
Copy link
Member

@mfisher87 mfisher87 commented Sep 18, 2024 •
edited
Loading

🚧 Work in progress 🚧

  • HARMONY_MIGRATION.md will contain my ongoing notes about the migration
  • There are some things mixed in here that arguably should be in different PRs. I'm keeping the commits atomic so we can cherry-pick it apart later if we prefer that.
  • ...

@github-actions GitHub Actions
Copy link

github-actions bot commented Sep 18, 2024 •
edited
Loading

Binder 👈 Launch a binder notebook on this branch for commit 79169a0

I will automatically update this comment whenever this PR is modified

Binder 👈 Launch a binder notebook on this branch for commit 01aff46

Binder 👈 Launch a binder notebook on this branch for commit 39b1318

Binder 👈 Launch a binder notebook on this branch for commit f65a07c

Binder 👈 Launch a binder notebook on this branch for commit bec4aa3

Binder 👈 Launch a binder notebook on this branch for commit ac916d6

Binder 👈 Launch a binder notebook on this branch for commit d67a7be

Binder 👈 Launch a binder notebook on this branch for commit 4021278

Binder 👈 Launch a binder notebook on this branch for commit 4379e1c

Binder 👈 Launch a binder notebook on this branch for commit 9638413

Binder 👈 Launch a binder notebook on this branch for commit 97e5dc6

Binder 👈 Launch a binder notebook on this branch for commit ba1b007

Binder 👈 Launch a binder notebook on this branch for commit 014d566

Binder 👈 Launch a binder notebook on this branch for commit f5b4214

Binder 👈 Launch a binder notebook on this branch for commit 0b2f704

@mfisher87 mfisher87 changed the title Replace ECS with Harmony [WIP] Replace ECS with Harmony Sep 18, 2024
@mfisher87 mfisher87 force-pushed the harmony branch 2 times, most recently from 4379e1c to 9638413 Compare September 26, 2024 16:14
@JessicaS11 JessicaS11 mentioned this pull request Sep 26, 2024
Closed
mfisher87 and others added 16 commits October 23, 2024 16:05
@mfisher87
Use CMR and EDL "UAT" environments
fe504ec 
User Acceptance Testing
@mfisher87
Add migration notes document
b627642
@mfisher87
Use NSIDC's UAT data provider for CMR
7ddb248
@mfisher87
Use envvar to pass EDL token to CMR to access restricted testing coll…
bb5eabd 
…ections

Removing the uat module should make it easy to find the rest of the code
we need to clean up.
@mfisher87
Document open questions and another assumption
7f693e3
@mfisher87
Add concept_id cached property to Query class
cf19a93 
We need this to talk to Harmony -- Harmony doesn't support the product
version as a query parameter, so we need to look up the real unique
identifier in advance.
@mfisher87
Remove redundant type info in docstring
6769f01
@mfisher87
Remove OBE docstring fragments
4d0e641
@mfisher87
Remove 1.x deprecation warning
609a07c
@mfisher87
Add note about Harmony's ICESat-2 live date
e39586c
@mfisher87
Answer open question about multiple Harmony APIs
6c7c862
@mfisher87
WIP
05c7ede 
Don't trust anything in here
@pre-commit-ci @mfisher87
[pre-commit.ci] auto fixes from pre-commit.com hooks
44fa0d1 
for more information, see https://pre-commit.ci
@mfisher87
Add a quick start to harmony migration
7cc7085
@mfisher87
Mention other ongoing icepyx efforts
239eaa0
@pre-commit-ci @mfisher87
[pre-commit.ci] auto fixes from pre-commit.com hooks
ba1b007 
for more information, see https://pre-commit.ci
@mfisher87 mfisher87 force-pushed the low-hanging-refactors branch from b75d000 to 2e7ac19 Compare October 23, 2024 22:06
@mfisher87 mfisher87 mentioned this pull request Oct 23, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mfisher87 @trey-stafford